-
Notifications
You must be signed in to change notification settings - Fork 83
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
New Donation Flow - Form Data Flow and Stripe Card submission #1311
Merged
dimitur2204
merged 83 commits into
1272-epic-new-version-of-the-donation-flow
from
1299-new-donatino-flow-data-flow-and-form-submission
Feb 1, 2023
Merged
New Donation Flow - Form Data Flow and Stripe Card submission #1311
dimitur2204
merged 83 commits into
1272-epic-new-version-of-the-donation-flow
from
1299-new-donatino-flow-data-flow-and-form-submission
Feb 1, 2023
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…04/frontend into payment-process-stripe
…ns page (#1303) * Fix maxWidth of Campaigns section on Homepage * Fix maxWidth for Campaigns section on Campaign page * Fix alignment of campaigns page * Fix spacings on campaigns section
…o be avaliable to both logged and unlogged users
* fixed scroll in admin campaign grid * allow edit of public donor name for all donations - #1206 * fixed: misleading names for bank upload * fixed typos valt vs vault * moved: vault grid actions as first column * fixed: editing of withdrawals and status change to successfully approved * added: withdrawnAmount column in Admin Campaign grid * forbid changes on succeeded withdrawal --------- Co-authored-by: quantum-grit <[email protected]>
…ation-flow' into 1299-new-donatino-flow-data-flow-and-form-submission
* Created basic POM for the Playwright tests - initial commit Added method and configurations Additional refactoring - methods, specs and gitignore Implemented Support page for E2E tests Fixed tests for Support page New changes on the tests * Created basic POM for the Playwright tests - initial commit Added method and configurations Additional refactoring - methods, specs and gitignore Implemented Support page for E2E tests Fixed tests for Support page New changes on the tests Fixes on E2E tests * Updated paths for the localization files for E2E * Merge branch '1194-feature-create-playwright-page-object-model-for-the-e2e-tests' of https://github.com/podkrepi-bg/frontend into 1194-feature-create-playwright-page-object-model-for-the-e2e-tests * Revert "Merge branch '1194-feature-create-playwright-page-object-model-for-the-e2e-tests' of https://github.com/podkrepi-bg/frontend into 1194-feature-create-playwright-page-object-model-for-the-e2e-tests" This reverts commit 3bb8a19, reversing changes made to b18b16b. * Updated Readme file * Fixed Sonar issues and updated Playwright version Additional fix * 1194 - init yarn project in the e2e folder and move out gitignore * 1194 - fix ts errors * 1194 - add e2e/.yarn to gitignore * 1194 - remove e2e/.yarn * 1194 - remove cached .yarn folder * 1194 - change GH Action workflow to now run the `test:e2e` * 1194 - change GH Action workflow to include artifacts of the reports and results * 1194 - fix home.page.ts ts errors * 1194 - remove hardcoded navigation to different environments * 1194 - add installation of dependencies for the e2e folder in the CI action * 1194 - fix translation on the homepage * 1194 - fix e2e script in the root folder * 1194 - add a tsconfig.build.json to exclude e2e while next is building * 1194 - add data based on the STAGING env variable * 1194 - resolve environment related tests * 1194 - remove uneccessasry `tsconfigPath` inside of the config * 1194 - put video to `off` by default * 1194 - add a clickCampaignByIndex method * 1194 - remove campaign specific test and changed them to index based * 1194 - add the index on the campaigns page list * 1194 - change the expiry date tests data to `04 / 42` * 1194 - add the index to the campaign card on the campaigns section on the home page * 1194 - add `playwright` as a dependency for the project * 1194 - add the changed `yarn.lock` * 1194 - fix the support action button click test to not click the card beforehand --------- Co-authored-by: Daniel Takev <[email protected]>
…low-data-flow-and-form-submission
dimitur2204
changed the title
1299 new donatino flow data flow and form submission
New Donation Flow - Form Data Flow and Stripe Card submission
Jan 31, 2023
dimitur2204
added
the
area: donations
Regarding something that includes the donation flow
label
Jan 31, 2023
@kachar Seems like without running |
dimitur2204
added
the
run tests
Allows running the tests workflows for forked repos
label
Jan 31, 2023
dimitur2204
deleted the
1299-new-donatino-flow-data-flow-and-form-submission
branch
February 1, 2023 17:47
1 task
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
area: donations
Regarding something that includes the donation flow
run tests
Allows running the tests workflows for forked repos
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #1299
It is tied with podkrepi-bg/api#425
Motivation and context
This includes the logic associated with the new way to make stripe donations. Check out #1299 for more information.
<Elements>
need to be wrapping the component that submits the paymentDonationFlowForm.tsx
in our case.<Elements>
throw an error, that initialized payment intent is then used for all future amount updates with the update requestcreate-stripe-payment
form which initializes the donation and then thestripe.confirmPayment
call is made to confirm the PaymentIntent and run any 3D Authentications that can be specific to each bankScreenshots:
Testing
Steps to test
You need to be on the podkrepi-bg/api#425 branch on the backend
campaigns/donation-v2/{slug}
Affected urls
campaigns/donation-v2/{slug}
Environment
New environment variables:
STRIPE_PUBLIC_KEY
: Shareable public key required for the @stripe to work